Skip to content

Remove closure in DefaultFunctionExecutor#2182

Merged
jviau merged 1 commit intoAzure:mainfrom
danielmarbach:function-executor
Jan 9, 2024
Merged

Remove closure in DefaultFunctionExecutor#2182
jviau merged 1 commit intoAzure:mainfrom
danielmarbach:function-executor

Conversation

@danielmarbach
Copy link
Copy Markdown
Contributor

@danielmarbach danielmarbach commented Jan 8, 2024 •
edited
Loading

Issue describing the changes in this PR

Removes the GetOrAdd closure in the DefaultFunctionExecutor

Pull request checklist

  • My changes do not require documentation changes
    • Otherwise: Documentation issue linked to PR
  • My changes should not be added to the release notes for the next release
    • Otherwise: I've added my notes to release_notes.md
  • My changes do not need to be backported to a previous version
    • Otherwise: Backport tracked by issue/PR #issue_or_pr
  • I have added all required tests (Unit tests, E2E tests)

Additional information

None

@jviau jviau self-requested a review January 9, 2024 18:49
@danielmarbach
Copy link
Copy Markdown
Contributor Author

danielmarbach commented Jan 9, 2024

Thanks!

@jviau jviau merged commit d9eb09c into Azure:main Jan 9, 2024
@jviau
Copy link
Copy Markdown
Contributor

jviau commented Jan 9, 2024

Thank you for the contribution!

@jviau jviau mentioned this pull request Jan 10, 2024
Merged
@danielmarbach danielmarbach deleted the function-executor branch January 12, 2024 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jviau jviau jviau approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@danielmarbach @jviau